Skip to content

fix: add Julia (.jl) support for code indexing with fallback chunking #8970

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

fix: add Julia (.jl) support for code indexing with fallback chunking #8970

wants to merge 1 commit into from

Conversation

@roomote
Copy link

@roomote roomote bot commented Nov 1, 2025 •
edited by ellipsis-dev bot
Loading

This PR attempts to address Issue #8966. Feedback and guidance are welcome.

Problem

Julia files (.jl) were not being indexed at all because:

  1. The .jl extension was not in the supported extensions list
  2. Without being in the extensions list, files were rejected before fallback chunking could be applied
  3. This meant Julia code was completely invisible to the codebase search feature

Solution

  • Added .jl to the supported extensions list in tree-sitter/index.ts
  • Added .jl to the fallbackExtensions array in shared/supported-extensions.ts
  • This enables Julia files to use fallback chunking for code indexing when tree-sitter parsing is not available

Testing

  • Added comprehensive test suite in parser.julia.spec.ts that verifies:
    • Julia files use fallback chunking
    • Small files below minimum size are handled correctly
    • Large files are chunked appropriately
    • Extension detection is case-insensitive
  • All existing tests continue to pass

Impact

  • Non-tree-sitter supported languages (like Julia) will now be properly indexed using fallback chunking
  • Users can now search their Julia codebase effectively
  • The same approach can be used to add support for other languages without tree-sitter parsers

Fixes #8966

Important

Add support for Julia files by enabling fallback chunking and updating supported extensions.

  • Behavior:
    • Add .jl to supported extensions in tree-sitter/index.ts and fallbackExtensions in shared/supported-extensions.ts.
    • Enables fallback chunking for Julia files when tree-sitter parsing is unavailable.
  • Testing:
    • New test suite parser.julia.spec.ts verifies fallback chunking for Julia files.
    • Tests include handling of small files, large files, and case-insensitive extension detection.
  • Impact:
    • Julia files are now indexed using fallback chunking, improving searchability.
    • Approach can be extended to other non-tree-sitter languages.

This description was created by Ellipsis for e91f7dc. You can customize this summary. It will automatically update as commits are pushed.

@roomote
fix: add Julia (.jl) support for code indexing with fallback chunking
e91f7dc 
- Add .jl extension to supported extensions list in tree-sitter/index.ts
- Add .jl to fallbackExtensions array in shared/supported-extensions.ts
- Add comprehensive tests for Julia file parsing and chunking
- Fixes issue where Julia files were not being indexed at all

Resolves #8966
@roomote roomote bot requested review from cte, jr and mrubens as code owners November 1, 2025 19:44
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working labels Nov 1, 2025
@roomote
Copy link
Author

roomote bot commented Nov 1, 2025 •
edited
Loading

See this task on Roo Code Cloud

✅ Review complete - no issues found. The implementation correctly adds Julia (.jl) support with fallback chunking and includes comprehensive test coverage.

Mention @roomote in a comment to trigger your PR Fixer agent and make changes to this pull request.

@roomote roomote bot mentioned this pull request Nov 1, 2025
Closed
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Nov 1, 2025
@daniel-lxs
Copy link
Member

daniel-lxs commented Nov 3, 2025

#8966 (comment)

@daniel-lxs daniel-lxs closed this Nov 3, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Nov 3, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mrubens mrubens Awaiting requested review from mrubens mrubens is a code owner

@cte cte Awaiting requested review from cte cte is a code owner

@jr jr Awaiting requested review from jr jr is a code owner

Assignees

No one assigned

Labels

bug Something isn't working Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Roo Code Roadmap
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] Code indexing fallback broken

4 participants

@daniel-lxs @hannesrudolph @roomote